Skip to content

refactor: remove setinputdefaultvalue as obsolete from actions - #2591

Open
maxy-shpfy wants to merge 1 commit into
masterfrom
07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions
Open

refactor: remove setinputdefaultvalue as obsolete from actions#2591
maxy-shpfy wants to merge 1 commit into
masterfrom
07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions

Conversation

@maxy-shpfy

@maxy-shpfy maxy-shpfy commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Description

Removes the redundant setInputDefaultValue action and replaces all usages with the existing setInputValue action, which serves the same purpose. This consolidates the two duplicate functions into a single action for setting an input's value.

Test Instructions

  1. Open a pipeline in the Editor
  2. Set a default value on a component input via the AI Chat tool bridge
  3. Verify the value is correctly applied and that undo/redo behavior works as expected

Additional Comments

setInputDefaultValue and setInputValue were functionally equivalent. This change removes setInputDefaultValue entirely and updates the CSOM bridge and useIOActions hook to use setInputValue directly.

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: 07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions/9e87791

maxy-shpfy commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

@maxy-shpfy
maxy-shpfy force-pushed the 07-29-chore_v2_-_sync_behavior_to_validation_fix_resolution branch from 6edb0bf to c1e8ed9 Compare July 29, 2026 18:58
@maxy-shpfy
maxy-shpfy force-pushed the 07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions branch from a6c3dd5 to 17dba89 Compare July 29, 2026 18:58
@maxy-shpfy
maxy-shpfy marked this pull request as ready for review July 29, 2026 19:43
@maxy-shpfy
maxy-shpfy requested a review from a team as a code owner July 29, 2026 19:43
@maxy-shpfy
maxy-shpfy changed the base branch from 07-29-chore_v2_-_sync_behavior_to_validation_fix_resolution to graphite-base/2591 July 30, 2026 17:17
@maxy-shpfy
maxy-shpfy force-pushed the graphite-base/2591 branch from c1e8ed9 to e9d694f Compare July 30, 2026 17:22
@maxy-shpfy
maxy-shpfy force-pushed the 07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions branch from 17dba89 to e9159a5 Compare July 30, 2026 17:22
@graphite-app
graphite-app Bot changed the base branch from graphite-base/2591 to master July 30, 2026 17:23
@maxy-shpfy
maxy-shpfy force-pushed the 07-29-refactor_remove_setinputdefaultvalue_as_obsolete_from_actions branch from e9159a5 to 9e87791 Compare July 30, 2026 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants